В целом хорошо)
На JavaFX сложновато учить Java, да и сам фреймворк не сильно используется.

Недочётов не нашёл, кроме:

1. Папки, где лежит код в проекте принято называть с "src/main/java/...", но это фигня - когда с мавеном разберёшься, он всё это будет делать за тебя.
2. Лучше разделять .fxml файлы и файлы кода. Их надо класть в "src/main/resources/..."

Если хочешь посмотреть пример моего кода - см. https://github.com/PospolitaNV/Teco
Но может быть немного сложновато) там использован Spring и Gradle (Вместо Maven)